Skip to content

refs platform/board#4052: add Typesense monitoring alerts and configuration for uptime checks and container checks#10

Merged
FabrizioCafolla merged 20 commits intomainfrom
feat/add-typesense-monitorings
Dec 11, 2025
Merged

refs platform/board#4052: add Typesense monitoring alerts and configuration for uptime checks and container checks#10
FabrizioCafolla merged 20 commits intomainfrom
feat/add-typesense-monitorings

Conversation

@FabrizioCafolla
Copy link
Copy Markdown
Member

@FabrizioCafolla FabrizioCafolla commented Dec 9, 2025

No description provided.

…ration for uptime checks and container checks
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds comprehensive Typesense monitoring capabilities to the Terraform module, including uptime checks for HTTP endpoints and container-level alerts for GKE environments.

  • Introduces a new typesense variable configuration with support for uptime checks and container monitoring
  • Implements two GKE container alert policies: pod restart monitoring and OOM (Out of Memory) killed detection
  • Adds changelog entry for version 0.6.0

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
variables.tf Adds new typesense variable with configuration for uptime checks, container checks (pod restarts, OOM), validation logic, and comprehensive documentation
typesense.tf Implements Typesense monitoring with uptime check module integration and two alert policies for pod restarts and OOM events
CHANGELOG.md Documents the addition of Typesense monitoring features in version 0.6.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread variables.tf Outdated
Comment thread typesense.tf Outdated
Comment thread typesense.tf Outdated
Comment thread CHANGELOG.md
Comment thread typesense.tf Outdated
FabrizioCafolla and others added 5 commits December 9, 2025 16:54
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@FabrizioCafolla
Copy link
Copy Markdown
Member Author

/improve

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread typesense.tf Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 8 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread variables.tf
Comment thread typesense.tf
Comment thread variables.tf Outdated
Comment thread variables.tf
Comment thread README.md
Comment thread typesense.tf
Comment thread typesense.tf
Comment thread typesense.tf Outdated
FabrizioCafolla and others added 4 commits December 11, 2025 11:29
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@filippolmt filippolmt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@FabrizioCafolla FabrizioCafolla merged commit 5107317 into main Dec 11, 2025
1 check passed
@FabrizioCafolla FabrizioCafolla deleted the feat/add-typesense-monitorings branch December 11, 2025 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants